Skip to content

Use schema in OpenAPI preview boundaries#673

Merged
RhysSullivan merged 1 commit intocodex/effect-lint-baselinefrom
codex/batch21-openapi-small-boundaries
May 6, 2026
Merged

Use schema in OpenAPI preview boundaries#673
RhysSullivan merged 1 commit intocodex/effect-lint-baselinefrom
codex/batch21-openapi-small-boundaries

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

@RhysSullivan RhysSullivan commented May 6, 2026

Summary

  • remove a redundant operationId cast
  • decode OpenAPI security scheme types through Schema
  • parse non-JSON request body test JSON with Schema.fromJsonString

Verification

  • bunx oxlint --format=unix packages/plugins/openapi/src/sdk/definitions.ts packages/plugins/openapi/src/sdk/preview.ts packages/plugins/openapi/src/sdk/non-json-body.test.ts
  • bun run --cwd packages/plugins/openapi typecheck
  • bun run --cwd packages/plugins/openapi test src/sdk/non-json-body.test.ts

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 6, 2026 •
edited
Loading

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs		 executor-cloud e9fd11a May 06 2026, 05:12 AM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 6, 2026 •
edited
Loading

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs		 executor-marketing e9fd11a Commit Preview URL

Branch Preview URL		 May 06 2026, 05:12 AM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 6, 2026

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@673

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@673

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@673

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@673

@executor-js/storage-core

npm i https://pkg.pr.new/@executor-js/storage-core@673

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@673

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@673

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@673

@executor-js/plugin-google-discovery

npm i https://pkg.pr.new/@executor-js/plugin-google-discovery@673

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@673

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@673

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@673

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@673

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@673

executor

npm i https://pkg.pr.new/executor@673

commit: e9fd11a

@RhysSullivan RhysSullivan merged commit 995c396 into codex/effect-lint-baseline May 6, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@RhysSullivan